Skip to content

Self trade#157

Merged
mihaimarcu2004 merged 5 commits into
mainfrom
self-trade
Jun 5, 2026
Merged

Self trade#157
mihaimarcu2004 merged 5 commits into
mainfrom
self-trade

Conversation

@mihaimarcu2004

@mihaimarcu2004 mihaimarcu2004 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

mihaimarcu2004 and others added 2 commits June 3, 2026 19:38
- sign_create_order: drop bogus cancel_all_market_index arg and fix
  argument order (self_trade_behavior/equality before skip_nonce)
- sign_cancel_all_orders: pass cancel_all_market_index through to the
  signer and default it to NIL_MARKET_INDEX (backward compatible)
- sign_modify_order: pass self_trade_equality_mode (was duplicating
  self_trade_behavior_mode)
- cancel_all_orders: default cancel_all_market_index to NIL_MARKET_INDEX
- rebuild + restore linux amd64/arm64 signer libs from lighter-go
  self-trade branch (they were dropped on this branch)
- wire darwin-amd64 dylib into the loader
- add self-trade and single-market cancel-all examples

Co-Authored-By: mihai <mihai2004marcu@gmail.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@mihaimarcu2004 mihaimarcu2004 merged commit c58758b into main Jun 5, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants